boxpointer: Don't trigger a relayout when updating arrow side

Changing the arrow side might need to reposition the boxpointer, however
if this happens during allocation, we don't need to trigger a new relayout since
we'd set the new allocation once _updateFlip's _reposition call is terminated,
otherwise if the position has changed, changing the boxpointer coordinates
will trigger a relayout anyways.

https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/576

diff --git a/js/ui/boxpointer.js b/js/ui/boxpointer.js
index 9939a5abfbbf0ccdd204d7bd16f14289d78c1ed9..2f4975a29e2990e93cebe00bd6fb326ab5b543ab 100644 (file)
--- a/js/ui/boxpointer.js
+++ b/js/ui/boxpointer.js
@@ -671,10 +671,6 @@ var BoxPointer = GObject.registerClass({
         if (this._arrowSide != arrowSide) {
             this._arrowSide = arrowSide;
             this._reposition(allocationBox);
-            Meta.later_add(Meta.LaterType.BEFORE_REDRAW, () => {
-                this.queue_relayout();
-                return false;
-            });
 
             this.emit('arrow-side-changed');
         }